Skip to content

Conversation

@jukkar
Copy link
Member

@jukkar jukkar commented Mar 20, 2019

The context_alloc_pkt() might run out of memory, and if that
happens we must not try to set the context pointer in it.

Signed-off-by: Jukka Rissanen [email protected]

The context_alloc_pkt() might run out of memory, and if that
happens we must not try to set the context pointer in it.

Signed-off-by: Jukka Rissanen <[email protected]>
@jukkar jukkar added bug The issue is a bug, or the PR is fixing a bug area: Networking labels Mar 20, 2019
@jukkar jukkar added this to the v1.14.0 milestone Mar 20, 2019
@jukkar jukkar requested a review from tbursztyka March 20, 2019 14:12
@jukkar jukkar requested a review from pfalcon as a code owner March 20, 2019 14:12
@jukkar jukkar requested a review from rveerama1 March 20, 2019 14:36
@jukkar
Copy link
Member Author

jukkar commented Mar 20, 2019

Actually this will conflict with #14670 which checks the null pointer. So if we apply that PR, then this one can be ignored.

@tbursztyka
Copy link
Contributor

#14670 is unlikely to be merged soon though.

@pfalcon
Copy link
Contributor

pfalcon commented Mar 20, 2019

#14670 is unlikely to be merged soon though.

How so? Please discuss it at the TSC together with #13618 .

@galak
Copy link
Contributor

galak commented Mar 20, 2019

How so? Please discuss it at the TSC together with #13618 .

I don't see any issue with merging #14670 as its a bug fix.

@galak galak merged commit bf8d391 into zephyrproject-rtos:master Mar 20, 2019
@jukkar jukkar deleted the net-context-pkt-alloc-failure branch February 29, 2024 08:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: Networking bug The issue is a bug, or the PR is fixing a bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants